Skip to content

fix spelling mistakes#5606

Merged
picklelo merged 2 commits into
reflex-dev:mainfrom
tartansandal:kjh/fix-spelling-mistakes
Jul 21, 2025
Merged

fix spelling mistakes#5606
picklelo merged 2 commits into
reflex-dev:mainfrom
tartansandal:kjh/fix-spelling-mistakes

Conversation

@tartansandal
Copy link
Copy Markdown
Contributor

While reading the code I noticed a couple of spelling mistakes being highlighted by my typos lsp. Running typos on the whole code base picked up some more. Hope this helps. 😅

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Greptile Summary

This PR contains spelling corrections across multiple files in the codebase, identified using the typos LSP tool. The changes include fixing common misspellings and typos in variable names, class names, comments, and test data. Some notable corrections include:

  • Variable/Class name fixes:

    • JavasciptKeyboardEventJavascriptKeyboardEvent
    • AddTaskProtcolAddTaskProtocol
    • _EMIITED_WARNINGS_EMITTED_WARNINGS
    • route_specifityroute_specificity
  • Documentation and test data fixes:

    • 'librarys' → 'libraries'
    • 'San Fransisco' → 'San Francisco'
    • 'granchild' → 'grandchild'
    • 'undercore' → 'underscore'

All changes are purely spelling corrections without any functional modifications to the code.

Confidence score: 5/5

  1. This PR is extremely safe to merge as it only fixes spelling mistakes
  2. All changes are straightforward spelling corrections with no functional impact
  3. Files needing attention: None - all changes are well-documented and clear

12 files reviewed, 2 comments
Edit PR Review Bot Settings | Greptile

Comment thread reflex/components/radix/themes/components/separator.py
Comment thread reflex/components/radix/themes/components/separator.py Outdated
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jul 21, 2025

CodSpeed Performance Report

Merging #5606 will not alter performance

Comparing tartansandal:kjh/fix-spelling-mistakes (605c7d7) with main (9114d21)

Summary

✅ 8 untouched benchmarks

Copy link
Copy Markdown
Contributor

@picklelo picklelo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice catches, thanks!

@picklelo picklelo merged commit fa41274 into reflex-dev:main Jul 21, 2025
40 of 41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants